Skip to content

Comments

Update tests to run on all pull requests#51

Merged
nhorton merged 5 commits intomainfrom
claude/update-pr-test-workflow-N9xH3
Jan 16, 2026
Merged

Update tests to run on all pull requests#51
nhorton merged 5 commits intomainfrom
claude/update-pr-test-workflow-N9xH3

Conversation

@nhorton
Copy link
Contributor

@nhorton nhorton commented Jan 16, 2026

…queue

The workflow now triggers on all pull requests so it shows as a required check, but both jobs skip their execution unless running in merge_group or workflow_dispatch. This ensures PRs show passing checks without running expensive tests on every PR update.

…queue

The workflow now triggers on all pull requests so it shows as a required
check, but both jobs skip their execution unless running in merge_group
or workflow_dispatch. This ensures PRs show passing checks without
running expensive tests on every PR update.
@nhorton nhorton added this pull request to the merge queue Jan 16, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 16, 2026
Add merge_group trigger and skip the CLA step when running in merge
queue. The CLA has already been verified on the PR, so no need to
re-check during merge queue validation.
Add section explaining how workflows are configured to use GitHub's
merge queue with skip patterns for required checks. This ensures
fast PR feedback while running expensive tests only in merge queue.
Explains workflow configuration, the skip pattern for required checks,
and how PRs get fast feedback while merge queue runs full validation.
@nhorton nhorton added this pull request to the merge queue Jan 16, 2026
@nhorton nhorton removed this pull request from the merge queue due to the queue being cleared Jan 16, 2026
@nhorton nhorton merged commit ef29d3d into main Jan 16, 2026
4 checks passed
@nhorton nhorton deleted the claude/update-pr-test-workflow-N9xH3 branch January 16, 2026 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants